Skip to content

Conversation

grahamc
Copy link
Member

@grahamc grahamc commented Jul 31, 2025

Motivation

Not intended to be merged directly. This PR is a convenience to show the diff between upstream Nix and Determinate Nix (the main branch).

Continuation of #4.

edolstra and others added 16 commits July 28, 2025 17:36
This change brings our Determinate Nix source repository in line with our standard branch naming.
One reason we used detsys-main was to avoid confusion in case the upstream NixOS/nix repository changed its default branch to main.
I don't anticipate that happening any time soon if ever, and so let's make the change.

Having this repo have a different branch name has a non-zero cost for humans ("oh right, different branch") and code to account for it.

Since we're shifting to internally run from `trunk()` (hi, jj) in more places, it's a "do it or don't" moment.
I choose to do it.
@grahamc grahamc requested a review from edolstra as a code owner July 31, 2025 17:14
@github-actions github-actions bot temporarily deployed to production July 31, 2025 17:14 Inactive
@DeterminateSystems DeterminateSystems locked as off-topic and limited conversation to collaborators Jul 31, 2025
@github-actions github-actions bot temporarily deployed to pull request July 31, 2025 18:20 Inactive
@github-actions github-actions bot temporarily deployed to production July 31, 2025 18:21 Inactive
@cole-h cole-h marked this pull request as draft August 1, 2025 14:26
This avoids problems with older versions of Nix that don't put the
caches in WAL mode. That's generally not a problem, until you do something like

  nix build --print-out-paths ... | cachix

which deadlocks because cachix tries to switch the caches to truncate
mode, which requires exclusive access. But the first process cannot
make progress because the cachix process isn't reading from the pipe.
With the migration to /nix/var/nix/builds we now have failing builds
when the derivation name is too long.
This change removes the derivation name from the temporary build to have
a predictable prefix length:

Also see: NixOS/infra#764
for context.

(cherry picked from commit 725a2f3)
(cherry picked from commit 7c3fd50)
@github-actions github-actions bot temporarily deployed to pull request September 17, 2025 18:13 Inactive
@github-actions github-actions bot temporarily deployed to production September 17, 2025 18:13 Inactive
don't include derivation name in temporary build directories
@github-actions github-actions bot temporarily deployed to production September 17, 2025 19:41 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 17, 2025 19:41 Inactive
xokdvium and others added 5 commits September 17, 2025 19:59
@github-actions github-actions bot temporarily deployed to pull request September 22, 2025 13:57 Inactive
@github-actions github-actions bot temporarily deployed to production September 22, 2025 13:57 Inactive
…expose-computefsclosure

libstore-c: add nix_store_get_fs_closure
@github-actions github-actions bot temporarily deployed to pull request September 23, 2025 14:01 Inactive
@github-actions github-actions bot temporarily deployed to production September 23, 2025 14:01 Inactive
RossComputerGuy and others added 5 commits September 23, 2025 12:55
This detects infinite recursion within the same thread (i.e. when a
thread waits on a thunk that it's already evaluating). This is done by
storing a thread ID in pending/awaited values.

It does not detect cycles between threads, e.g. in `rec { x = y; y =
x; }` if `x` and `y` are marked as pending by different threads. But
those are much less likely in practice.
Upstream changes to prepare for the nario command
@github-actions github-actions bot temporarily deployed to pull request September 25, 2025 16:28 Inactive
@github-actions github-actions bot temporarily deployed to production September 25, 2025 16:28 Inactive
…sion-v2

Fix infinite recursion detection
@github-actions github-actions bot temporarily deployed to production September 26, 2025 09:30 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 26, 2025 09:30 Inactive
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants